Remove GTK_FILE_CHOOSER_ACTION_CREATE_FOLDER

We don't need a whole separate action, now that the file chooser widget
can create folders; we can create use SELECT_FOLDER and create one.

Remove GtkFileChooser:local-only

Now that the whole API goes through GFile we don't have the weird split
between local-only and non-local-only modes.

Remove GtkFileChooser:show-hidden

This is a user setting and a user action, not something that ought to be
programmatically set via the file selection API.

Remove overwrite confirmation machinery from GtkFileChooser

Overwrite confirmation should not be optional, and it should not loop
into application code to create their own dialog and user response.

Drop GtkFileChooser:extra-widget

We have "choices" as a more rational (and portable) API; additionally,
the ownership semantics of the extra widget property getter are a hack.

Remove filename/URI API from GtkFileChooser

GtkFileChooser's API predates GIO by a few years, so it started off with
filenames and URI as character arrays. After introducing GIO as a
dependency, the API included GFile-based entry points.

It's much more appropriate to use GFile everywhere, as we want to
encourage people to use GIO instead of passing random bytes to low level
POSIX API.

See: #2455

Merge branch 'wip/otte/dnd' into 'master'

stuff

See merge request GNOME/gtk!1460

Remove GdkSeatGrabPrepareFunc from headers

It is not used in public api anymore.

Add a warning when a grab fails

This reveals that the grabs for popover menus
always fail now, causing the menus to not properly
hide.

tests: Guard variables with necessary macros

treeviewcolumn: Inline variable into return_if_fail()

testsuite: Don't use g_return_if_fail()

Use g_assert()

testsuite: Don't use return_if_fail()

Use assert() instead

treeview: Guard code by right compiler macros

scrolledwindow: Inline variable into return_if_fail()

iconview: Inline variables into return_if_fail()

levelbar: Guard function by right compiler macros

filesystemmodel: Guard variable with correct compiler flags

combobox: Inline variables into return_if_fail()

comboboxtext: Inline functions into return_if_fail()

device: Inline function into return_if_fail()

drop: Guard variable with correct compiler flags

surface: Inline function

Fixes compile error with -DG_DISBALE_CHECKS

main: Don't synthesize crossing events when nothing changed

This was especially bad because it was confusing the event controllers
so much, they'd emit leave + enter events every time the mouse moved.

stackswitcher: Use GdkDropControllerMotion

... and use one controller per button instead of using it on the
switcher and then going through lots of pain attempting to find the
right button for the location under the mouse.

Port simple cases to GtkDropControllerMotion

testsuite: Check proper notify emissions, too

This is in particular relevant for the ::is-focus property, because
updating that one doesn't cause enter/leave events.

But it also checks that notify and enter/leave happen in the right
order.

Add GtkDropControllerMotion

eventcontrollermotion: Fix docs

gtk: Bubble drag events like motion events

Emit crossing events - with a new GTK_CROSSING_DROP type - like we do
for motion events. There is no more special casing for them.

Note that the gesture has not been updated yet, so some obscure behavior
may occur.

events: Use GdkDrop as event sequence

This allows treating drop events like touch events, which GTK groups by
event sequence.

It's a bit ugly that we just case the GdkDrop pointer, but event
sequences are only meant to be unique pointer ids, so it's fine.

widget: Simplify adjust_allocation()

And in particular, only do it if the widget doesn't use ALIGN_FILL.

This avoids lots of measuring in the common case and speeds up
size_allocate() by about 25%.

And because size_allocate() is the bottleneck in the fishbowl, this also
gets ~25% more fishies.

widget: Pull margin computation out of adjust_allocation

It's way cheaper to just do it.

Also simplifies adjust_size_allocation a lot.

sizerequest: Stop clamping for_size to natural size

Widgets should be given the actual size they will be allocated, so they
can do something with it.

If they want to clamp themselves to their natural size, nothing's
stopping them, they know their natural size after all.

widget: Don't check for natives

It's the native's job to request a 1px x 1px size, not the job of
gtk_widget_size_allocate()

Also saves 10% of size_allocate() time because checking for an interface
is really expensive.

Ensure all natives request at least a 1px wide surface

FIXME: Is this necessary?

Could the surfaces just clamp to 1x1 themselves?
We recently declared that surfaces can decide on whatever size they want
so natives need to inspect the size they requested anyway.

Merge branch 'color-float-fix' into 'master'

color chooser: Fix fallout from floatification

See merge request GNOME/gtk!1459

color chooser: Fix fallout from floatification

We are using (dddd) variants to store colors in variants,
which is dangerous now that GdkRGBA members are just floats.

Avoid passsing the GdkRGBA members directly to any varargs
functions.

Merge branch 'wip/xdg-popup-layout-no-op' into 'master'

gdk/wayland: Avoid relayout with the same properties

See merge request GNOME/gtk!1457

Merge branch 'wip/gdkpopuplayout-section' into 'master'

docs: Add GdkPopupLayout to gdk4-sections.txt

See merge request GNOME/gtk!1458

docs: Add GdkPopupLayout to gdk4-sections.txt

gdk/wayland: Avoid relayout with the same properties

When a popup is already showing, and gdk_surface_present_popup() is
called, if the layout didn't change, we're not really interested in
relayouting.

In the future, we'll be able to get notified if position of the popup
would change by some environmental changes, but until then, just don't
support it.

gdk/popup-layout: Remove leftover struct field

The layouts never "seal", as they did in an earlier revision, so remove
the seal field.

Merge branch 'wip/otte/for-master' into 'master'

surface: Don't take a display argument in gdk_surface_new_popup()

See merge request GNOME/gtk!1456

surface: Don't take a display argument in gdk_surface_new_popup()

The display can be taken from the surface, it must not be different.

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!1455

transform: Don't crash for gsk_transform_transform (id, id)

See attached tests

transform: Make sure the identity transform is equal to NULL

x11: When clearing old Drop, emit LEAVE event

This can happen when the old DND operation died (like due to a crash or
a broken XWayland compositor.

gdk: Make DRAG_ENTER event take x/y coordinates

Make it mirror the behavior of ENTER/LEAVE events.

transform: Add optimization for common case

Transforming identity by an other transform does not mean we need to
painstakingly apply the individual steps of other to construct a new
transform, it means we can just return other.

Or in math terms:
  I * B = B
so just return B.

Update POTFILES.in

profiling: Avoid one extra printf

We already format the message, no need to use
printf again to combine that with the kind string.

profiling: Avoid criticals

With events no longer GObjects, the type class is
longer around for peeking.

Merge branch 'readonly-events-1' into 'master'

Redo events

See merge request GNOME/gtk!1443

docs: Remove some no-longer existing api

Stop exporting gtk_get_event_widget

This is non-essential convenience API, and we
don't really expose events to applications anymore.

gtk-demo: Stop using gtk_get_event_widget

This is convenience api that will be going away.

gdk: Drop event structs from the headers

All events are GdkEvents now.

gtk: Stop using GtkEventButton

Same as the previous commit. All events are
just GdkEvents now.

gtk: Stop using GdkEventKey

We should just use GdkEvent, instead of taking
a GdkEventKey, and then cast it everywhere.

a11y: drop unused code

We are no longer using key snoopers.

motion controller: Match focus event propagation

Make the crossing event generation for pointer events
match what we do for focus now.

Keep more event controller api private

We don't want to expose the GtkCrossingData struct, and manually
feeding events to event controllers is not something we want to
encourage, going forward.

Stop exporting check_event_sanity

Bring back im context focus-in/out

The key controller still needs to track focus,
in order to emit these signals when required.

Go back to ::enter/::leave for pointer changes

These signals are behaving a little differently from
what ::focus-in/::focus-out used to do.

Split off GtkEventControllerFocus

Split the focus tracking into a separate
GtkEventControllerFocus, and change the API one more time.
We are back to having ::focus-in and ::focus-out signals.

Update all users.

wip: Add more information to crossing events

Add fields for direct descendents to GtkCrossingData,
and populate them when emitting focus change events.

Also add accessors for these fields to GtkEventControllerKey,
and verify that they are set properly in the focus test.

Not done yet: Do the same for pointer crossing events.

win32: Remove some leftover event struct access

And leftover old event api calls.

wayland: Fix a release build warning

Update the focus test

This needs an update to handle the new focus-change signal.

Update event docs section

Clean up GdkEventType docs

Some event struct packing improvements

Rearrange a few things, and move some booleans
into the Any struct, by using a bitfield there.

Some more cleanup could be done - the flags field
with its PENDING and FLUSHED members appears
entirely unused. Nobody is setting those flags.

Streamline event structs

Use proper types (GdkModifierType, double), and drop
some unused fields (send_event, display).

x11: Remove some leftover event struct access

x11: Stop using the send_event event field

The only use of this field is printing it out in a
debug message. We are going to drop it.

Always deliver focus events to toplevels

Its was GTK expects. This change gets rid of the "Ignoring an
unexpected focus event from GDK on a non-toplevel surface."
warning.

Drop gtk_widget_event from API

We don't want events to be injected randomly from
the outside. Plus, there's no way to create such
events from the outside now.

events: reorganize getters

Restructure the getters for event fields to
be more targeted at particular event types.

Update all callers, and replace all direct
event struct access with getters.

As a side-effect, this drops some unused getters.

Drop gdk_set/get_show_events

We don't need a separate api to turn on debugging
for events.

Strip const from GdkEvent

Events are refcounted structs, and we generally don't
pass these as const.

events: Drop all setters

GdkEvent is now truly readonly.

events: Drop target and related target

GTK no longer uses this, so we can drop it.

main: Stop calling gdk_event_set_target

GTK is no longer relying on this.

Stop using gdk_event_get_target

We can now get the target widget from the gesture
that we are using to find the event in the first
place.

gesture: Keep the target widget for events

We already store the events; keep the target widget
in addition. This is a step towards getting rid of
gdk_event_get_target.

eventcontroller: Make the target widget available

Make it possible for event controllers to obtain
the target widget during handle_event.

Explicitly pass the target to handle_event

Pass the event propagation target explicitly down to
the event controllers. This is a step towards getting
rid of gdk_event_set_target.

Stop looking at the related target for filtering

We are now sending crossing events (which are the only ones
where a related target makes sense) via handle_crossing.

Reinstate filtering for crossing events

The event propagation limit should apply to crossing events
as well.

Make crossing events handled the same way

New focus change handling

Instead of relying on gdk's antiquated crossing events,
create a new GtkCrossingData struct that contains the
actual widgets, and a new event controller vfunc that
expects this struct. This also saves us from making sense
of X's crossing modes and details, and makes for a
generally simpler api.

The ::focus-in and ::focus-out signals of GtkEventControllerKey
have been replaced by a single ::focus-change signal that
takes GtkCrossingData as an argument. All callers have
been updated.

wip: scrolledwindow stop using targets

main: Drop gtk_get_event_target

This is no longer used.

tooltip: stop using gtk_get_event_target

Stop using gtk_get_event_target

This is just a thin wrapper around gdk_event_get_target,
so use that directly.

Pass translated coordinates outside the event

We want to make events readonly, so stop translating
their coordinates and instead pass the translated
coordinates separately, when propagating events.

Drop GDK_NOTHING

Events of type GDK_NOTHING are good for nothing.